Skip to content

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Aug 28, 2025

Backports the following commits to 9.1:

Most upper bounds files for transport version are for an already
branched version. For those, the upper bound must be within the base id
for that branch, and validation exists to ensure the highest id within a
branch is in the upper bound file for that branch. But for main the base
is always different. This commit adds validation to ensure that the
highest transport version id exists in _some_ upper bound file.

Note that while we could try to identify which upper bound file belongs
to main, this is tricky when validation runs in non-main branches. This
check is just as good, just a little less specific than "it should exist
in x.y upper bound file".
@rjernst rjernst added the :Core/Infra/Core Core issues without another label label Aug 28, 2025
@rjernst rjernst requested a review from a team as a code owner August 28, 2025 16:36
@rjernst rjernst added >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Core/Infra Meta label for core/infra team labels Aug 28, 2025
@elasticsearchmachine elasticsearchmachine merged commit 272e49d into elastic:9.1 Aug 28, 2025
35 checks passed
@rjernst rjernst deleted the backport/9.1/pr-133689 branch August 28, 2025 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v9.1.4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants